Don't use the client-side shortcut for get_uuid calls -- we need to keep

handles and UUIDs conceptually separate.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

diff --git a/tools/libxen/include/xen_user.h b/tools/libxen/include/xen_user.h
index cce9ba8302d26568e1ef0f5e51572f68b9d9299d..9669e071163b1be1f4980c04839f789bd03185fc 100644 (file)
--- a/tools/libxen/include/xen_user.h
+++ b/tools/libxen/include/xen_user.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2006, XenSource Inc.
+ * Copyright (c) 2006-2007, XenSource Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -24,8 +24,8 @@
 
 
 /*
- * The user class. 
- *  
+ * The user class.
+ * 
  * A user of the system.
  */
 

diff --git a/tools/libxen/src/xen_console.c b/tools/libxen/src/xen_console.c
index a5a5ad2bb223bde6c2d759e99e04db0f395b5a79..9d0764a2d662337eacea6fbb1b45a3d6f392bed3 100644 (file)
--- a/tools/libxen/src/xen_console.c
+++ b/tools/libxen/src/xen_console.c
@@ -284,6 +284,15 @@ xen_console_get_all(xen_session *session, struct xen_console_set **result)
 bool
 xen_console_get_uuid(xen_session *session, char **result, xen_console console)
 {
-    *result = session->ok ? xen_strdup_((char *)console) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = console }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("console.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_crashdump.c b/tools/libxen/src/xen_crashdump.c
index 2d3840896e8592f2fc317dd1d7ae56bca860cc32..5d8f70a2ea2a5380bebf67ad8a68e62a8813238d 100644 (file)
--- a/tools/libxen/src/xen_crashdump.c
+++ b/tools/libxen/src/xen_crashdump.c
@@ -177,6 +177,15 @@ xen_crashdump_get_all(xen_session *session, struct xen_crashdump_set **result)
 bool
 xen_crashdump_get_uuid(xen_session *session, char **result, xen_crashdump crashdump)
 {
-    *result = session->ok ? xen_strdup_((char *)crashdump) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = crashdump }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("crashdump.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_host.c b/tools/libxen/src/xen_host.c
index 204664ae3c0fe2442826c8bd3fcb7dad85347c91..64badac031a0ce2764a13dfb1b42c91808cd939e 100644 (file)
--- a/tools/libxen/src/xen_host.c
+++ b/tools/libxen/src/xen_host.c
@@ -754,6 +754,15 @@ xen_host_get_all(xen_session *session, struct xen_host_set **result)
 bool
 xen_host_get_uuid(xen_session *session, char **result, xen_host host)
 {
-    *result = session->ok ? xen_strdup_((char *)host) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = host }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("host.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_host_cpu.c b/tools/libxen/src/xen_host_cpu.c
index 1494c10191227b1b05905941356173157c182c66..c03241dc84ddeccbc2642a17a877ab8c8863e6a2 100644 (file)
--- a/tools/libxen/src/xen_host_cpu.c
+++ b/tools/libxen/src/xen_host_cpu.c
@@ -282,6 +282,15 @@ xen_host_cpu_get_all(xen_session *session, struct xen_host_cpu_set **result)
 bool
 xen_host_cpu_get_uuid(xen_session *session, char **result, xen_host_cpu host_cpu)
 {
-    *result = session->ok ? xen_strdup_((char *)host_cpu) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = host_cpu }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("host_cpu.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_host_metrics.c b/tools/libxen/src/xen_host_metrics.c
index bc621fbaae2b8ebfe6e0ab9c8877c01fa01be3b0..203f25ed0a4f3d5e59c593093b53e97edf50fcea 100644 (file)
--- a/tools/libxen/src/xen_host_metrics.c
+++ b/tools/libxen/src/xen_host_metrics.c
@@ -176,6 +176,15 @@ xen_host_metrics_get_all(xen_session *session, struct xen_host_metrics_set **res
 bool
 xen_host_metrics_get_uuid(xen_session *session, char **result, xen_host_metrics host_metrics)
 {
-    *result = session->ok ? xen_strdup_((char *)host_metrics) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = host_metrics }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("host_metrics.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_network.c b/tools/libxen/src/xen_network.c
index 2351404705317beb9e9a541fcc50acf6b8228515..723616204cf7df17085f7abbff843f71eecc3183 100644 (file)
--- a/tools/libxen/src/xen_network.c
+++ b/tools/libxen/src/xen_network.c
@@ -285,6 +285,15 @@ xen_network_get_all(xen_session *session, struct xen_network_set **result)
 bool
 xen_network_get_uuid(xen_session *session, char **result, xen_network network)
 {
-    *result = session->ok ? xen_strdup_((char *)network) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = network }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("network.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_pbd.c b/tools/libxen/src/xen_pbd.c
index 1bdcd785fa0fe9f0b424762cfa66ef03be32b359..2b8bbd5b44b89f2a34defd753f7369c539fc8795 100644 (file)
--- a/tools/libxen/src/xen_pbd.c
+++ b/tools/libxen/src/xen_pbd.c
@@ -235,6 +235,15 @@ xen_pbd_get_all(xen_session *session, struct xen_pbd_set **result)
 bool
 xen_pbd_get_uuid(xen_session *session, char **result, xen_pbd pbd)
 {
-    *result = session->ok ? xen_strdup_((char *)pbd) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = pbd }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("PBD.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_pif.c b/tools/libxen/src/xen_pif.c
index f710acef4b671ec24398c5e299de11c7edd7a6e9..9dfaec7b519bae6cb30ddbd705e0d218198475bc 100644 (file)
--- a/tools/libxen/src/xen_pif.c
+++ b/tools/libxen/src/xen_pif.c
@@ -366,6 +366,15 @@ xen_pif_get_all(xen_session *session, struct xen_pif_set **result)
 bool
 xen_pif_get_uuid(xen_session *session, char **result, xen_pif pif)
 {
-    *result = session->ok ? xen_strdup_((char *)pif) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = pif }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("PIF.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_pif_metrics.c b/tools/libxen/src/xen_pif_metrics.c
index 5826001731d4a7381cfa0d2b54bfdc3359329129..c1413442cad1dda0a085fd29b1304fb44a9c0215 100644 (file)
--- a/tools/libxen/src/xen_pif_metrics.c
+++ b/tools/libxen/src/xen_pif_metrics.c
@@ -176,6 +176,15 @@ xen_pif_metrics_get_all(xen_session *session, struct xen_pif_metrics_set **resul
 bool
 xen_pif_metrics_get_uuid(xen_session *session, char **result, xen_pif_metrics pif_metrics)
 {
-    *result = session->ok ? xen_strdup_((char *)pif_metrics) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = pif_metrics }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("PIF_metrics.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_sr.c b/tools/libxen/src/xen_sr.c
index c3650f26a036ae8d00938a5e4e44d572e82b45d5..86538241a18927ddf6c902670bb739edfef2564c 100644 (file)
--- a/tools/libxen/src/xen_sr.c
+++ b/tools/libxen/src/xen_sr.c
@@ -388,6 +388,15 @@ xen_sr_get_all(xen_session *session, struct xen_sr_set **result)
 bool
 xen_sr_get_uuid(xen_session *session, char **result, xen_sr sr)
 {
-    *result = session->ok ? xen_strdup_((char *)sr) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = sr }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("SR.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_user.c b/tools/libxen/src/xen_user.c
index 35b80e2f66384701556cc51415a455234f5b31f6..b4dde9d74e2129b3b6d6d33796f26036a50d76d5 100644 (file)
--- a/tools/libxen/src/xen_user.c
+++ b/tools/libxen/src/xen_user.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2006, XenSource Inc.
+ * Copyright (c) 2006-2007, XenSource Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -196,6 +196,15 @@ xen_user_set_fullname(xen_session *session, xen_user user, char *fullname)
 bool
 xen_user_get_uuid(xen_session *session, char **result, xen_user user)
 {
-    *result = session->ok ? xen_strdup_((char *)user) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = user }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("user.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vbd.c b/tools/libxen/src/xen_vbd.c
index 893bf70958f32ac61c0cc80aedde97cac9bc1f70..cd63a4b52e99404c014b17e8d28c310d9aeeebb6 100644 (file)
--- a/tools/libxen/src/xen_vbd.c
+++ b/tools/libxen/src/xen_vbd.c
@@ -591,6 +591,15 @@ xen_vbd_get_all(xen_session *session, struct xen_vbd_set **result)
 bool
 xen_vbd_get_uuid(xen_session *session, char **result, xen_vbd vbd)
 {
-    *result = session->ok ? xen_strdup_((char *)vbd) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vbd }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VBD.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vbd_metrics.c b/tools/libxen/src/xen_vbd_metrics.c
index f0b221f65e3f15e7114ea45d599ce9348436a00d..fb5395d3096f0f02c7c199f38695bc42c2c638b8 100644 (file)
--- a/tools/libxen/src/xen_vbd_metrics.c
+++ b/tools/libxen/src/xen_vbd_metrics.c
@@ -176,6 +176,15 @@ xen_vbd_metrics_get_all(xen_session *session, struct xen_vbd_metrics_set **resul
 bool
 xen_vbd_metrics_get_uuid(xen_session *session, char **result, xen_vbd_metrics vbd_metrics)
 {
-    *result = session->ok ? xen_strdup_((char *)vbd_metrics) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vbd_metrics }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VBD_metrics.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vdi.c b/tools/libxen/src/xen_vdi.c
index 10f3fc4066596412ff0f42d89cb828c569d1aadb..91cc30602185cccf42b984623fbb7a198dbba2d1 100644 (file)
--- a/tools/libxen/src/xen_vdi.c
+++ b/tools/libxen/src/xen_vdi.c
@@ -555,6 +555,15 @@ xen_vdi_get_all(xen_session *session, struct xen_vdi_set **result)
 bool
 xen_vdi_get_uuid(xen_session *session, char **result, xen_vdi vdi)
 {
-    *result = session->ok ? xen_strdup_((char *)vdi) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vdi }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VDI.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vif.c b/tools/libxen/src/xen_vif.c
index a139a83bc3836dbcabfd8cbdd30b9f7513efbc4a..9a44129e4f7dfb9435c8166d52d36d8616fabef7 100644 (file)
--- a/tools/libxen/src/xen_vif.c
+++ b/tools/libxen/src/xen_vif.c
@@ -542,6 +542,15 @@ xen_vif_get_all(xen_session *session, struct xen_vif_set **result)
 bool
 xen_vif_get_uuid(xen_session *session, char **result, xen_vif vif)
 {
-    *result = session->ok ? xen_strdup_((char *)vif) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vif }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VIF.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vif_metrics.c b/tools/libxen/src/xen_vif_metrics.c
index b3b2a79f59da1a1116c16527b38118ae242cc53c..7c101f89337509dd4768aff152fb6d1ac366f876 100644 (file)
--- a/tools/libxen/src/xen_vif_metrics.c
+++ b/tools/libxen/src/xen_vif_metrics.c
@@ -176,6 +176,15 @@ xen_vif_metrics_get_all(xen_session *session, struct xen_vif_metrics_set **resul
 bool
 xen_vif_metrics_get_uuid(xen_session *session, char **result, xen_vif_metrics vif_metrics)
 {
-    *result = session->ok ? xen_strdup_((char *)vif_metrics) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vif_metrics }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VIF_metrics.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vm.c b/tools/libxen/src/xen_vm.c
index f80916188ee4db13571192d522d7c6048b502e37..f29d34f7eb72362d9659bcf7153b4e7a03d31c08 100644 (file)
--- a/tools/libxen/src/xen_vm.c
+++ b/tools/libxen/src/xen_vm.c
@@ -1608,6 +1608,15 @@ xen_vm_get_all(xen_session *session, struct xen_vm_set **result)
 bool
 xen_vm_get_uuid(xen_session *session, char **result, xen_vm vm)
 {
-    *result = session->ok ? xen_strdup_((char *)vm) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vm }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VM.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vm_guest_metrics.c b/tools/libxen/src/xen_vm_guest_metrics.c
index f18966c46aae1ee1c247e885cddf653dceb2c568..0e60926babdcc9ff50cd9d95a49d010ef8eb77d4 100644 (file)
--- a/tools/libxen/src/xen_vm_guest_metrics.c
+++ b/tools/libxen/src/xen_vm_guest_metrics.c
@@ -265,6 +265,15 @@ xen_vm_guest_metrics_get_all(xen_session *session, struct xen_vm_guest_metrics_s
 bool
 xen_vm_guest_metrics_get_uuid(xen_session *session, char **result, xen_vm_guest_metrics vm_guest_metrics)
 {
-    *result = session->ok ? xen_strdup_((char *)vm_guest_metrics) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vm_guest_metrics }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VM_guest_metrics.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vm_metrics.c b/tools/libxen/src/xen_vm_metrics.c
index ae20899e06cd4e3fa075380ad4e9d95c739caf40..11e421282bbc4cb1f26f3f3e70df95d0d6747da8 100644 (file)
--- a/tools/libxen/src/xen_vm_metrics.c
+++ b/tools/libxen/src/xen_vm_metrics.c
@@ -198,6 +198,15 @@ xen_vm_metrics_get_all(xen_session *session, struct xen_vm_metrics_set **result)
 bool
 xen_vm_metrics_get_uuid(xen_session *session, char **result, xen_vm_metrics vm_metrics)
 {
-    *result = session->ok ? xen_strdup_((char *)vm_metrics) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vm_metrics }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VM_metrics.get_uuid");
     return session->ok;
 }

diff --git a/tools/libxen/src/xen_vtpm.c b/tools/libxen/src/xen_vtpm.c
index 2b695f6ee2c5340b1c99ed9e948ff0eae756de3a..28d849ae1cffa73c7fd863a18fa62b67e6f8a0e4 100644 (file)
--- a/tools/libxen/src/xen_vtpm.c
+++ b/tools/libxen/src/xen_vtpm.c
@@ -182,6 +182,15 @@ xen_vtpm_get_backend(xen_session *session, xen_vm *result, xen_vtpm vtpm)
 bool
 xen_vtpm_get_uuid(xen_session *session, char **result, xen_vtpm vtpm)
 {
-    *result = session->ok ? xen_strdup_((char *)vtpm) : NULL;
+    abstract_value param_values[] =
+        {
+            { .type = &abstract_type_string,
+              .u.string_val = vtpm }
+        };
+
+    abstract_type result_type = abstract_type_string;
+
+    *result = NULL;
+    XEN_CALL_("VTPM.get_uuid");
     return session->ok;
 }